Skip to content

fix(kilo-pass): keep first month bonus for new subscribers#3141

Merged
iscekic merged 1 commit intomainfrom
feat/kilo-pass-bonus-cutoff
May 8, 2026
Merged

fix(kilo-pass): keep first month bonus for new subscribers#3141
iscekic merged 1 commit intomainfrom
feat/kilo-pass-bonus-cutoff

Conversation

@iscekic
Copy link
Copy Markdown
Contributor

@iscekic iscekic commented May 8, 2026

Summary

Update Kilo Pass monthly promo rules so first-time subscribers always receive the first-month 50% bonus, while the second-month 50% bonus is only grandfathered for subscriptions started before 2026-05-09T00:00:00Z. The Kilo Pass subscription and detail views now show first-month promo copy for eligible users and only show second-month copy when the grandfather rule applies.

Verification

N/A - no manual UI verification performed; this is a rules/copy update covered with targeted automated checks.

Visual Changes

N/A

Reviewer Notes

Review the cutoff semantics around started_at: the month-two 50% bonus requires started_at strictly before 2026-05-09T00:00:00Z, while month one remains 50% for first-time subscribers regardless of start time.

@iscekic iscekic self-assigned this May 8, 2026
@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 8, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (7 files)
  • apps/web/src/components/profile/kilo-pass/KiloPassSubscribeCard.tsx
  • apps/web/src/components/subscriptions/kilo-pass/KiloPassDetail.tsx
  • apps/web/src/lib/kilo-pass/bonus.test.ts
  • apps/web/src/lib/kilo-pass/bonus.ts
  • apps/web/src/lib/kilo-pass/constants.ts
  • apps/web/src/routers/kilo-pass-router.test.ts
  • apps/web/src/routers/kilo-pass-router.ts

Reviewed by gpt-5.5-2026-04-23 · 513,048 tokens

@iscekic iscekic enabled auto-merge (squash) May 8, 2026 16:31
@iscekic iscekic merged commit 1d1539f into main May 8, 2026
17 checks passed
@iscekic iscekic deleted the feat/kilo-pass-bonus-cutoff branch May 8, 2026 17:12
@lambertjosh
Copy link
Copy Markdown
Contributor

As-is (Cutoff of 2026-05-09):
image

Cutoff in the past:
image

@lambertjosh
Copy link
Copy Markdown
Contributor

lambertjosh commented May 8, 2026

LGTM, tried to do a more thorough review and test given last PR.

@lambertjosh
Copy link
Copy Markdown
Contributor

Note I did not test that this affects actual credit amount given in 2nd month, wasn't sure how to easily test that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants